size group: Make not subclassable

Part of being more explicit about what
we allow to be subclassed and what not.

password entry: Make not subclassable

Part of being more explicit about what
we allow to be subclassed and what not.

Padding review

Ensure that the class structs of all subclassable
types have sufficient padding (standardizing on 8
slots, here).

GtkBox
GtkButton
GtkDrawingArea
GtkFixed
GtkFrame

Revert "wayland: Make popups work more than once"

This reverts commit e74c655016982272b72dff318b11602b0d5b099a.

message dialog: Make not subclassable

Part of being more explicit about what
we allow to be subclassed and what not.

Padding review

Ensure that the class structs of all subclassable
types have sufficient padding (standardizing on 8
slots, here).

GtkApplication
GtkWidget
GtkContainer
GtkWindow
GtkDialog
GtkApplicationWindow
GtkToolItem
GtkBin

Cosmetic changes

Use new-style padding for GdkContentProviderClass.

Trivial: Remove a few +'s

Merge branch 'frame-pixel-counter' into 'master'

profiling: Add a counter for pixels drawn per frame

See merge request GNOME/gtk!855

wayland: Make popups work more than once

The change to keep some server resources around
until destroy was causing us to not recreate
the right things when a surface is hidden and
then shown again. Make sure to recreate everything.

widget-factory: menu buttons can't have children

GtkMenuButton is no longer a container, so we
can't pack children in it.

Merge branch 'wayland-surface-unmap' into 'master'

wayland: Keep some resources until destroy

Closes #1485

See merge request GNOME/gtk!857

wayland: Keep some resources until destroy

The Wayland backend was dropping _all_ serverside
resources on hide, which is too early e.g. for
GtkGLArea which wants to use egl resources to
unload textures on unrealize.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1485

Merge branch 'button-clicked' into 'master'

Drop gtk_button_clicked

See merge request GNOME/gtk!856

Drop gtk_button_clicked

This is not an api we want to propagate anymore.
If you need to, you can still emit the "clicked"
action signal on a button using g_signal_emit_by_name.

demos: Stop using gtk_button_clicked

gtk: Stop using gtk_button_clicked

This function is going away.

profiling: Add a counter for pixels drawn per frame

This number clearly shows the recently discovered
"full redraws" problem.

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

Closes #1900

See merge request GNOME/gtk!854

build: Convert CSS parser tests to single meson tests

glcontext: Store the updated area in the right place

We need to store the region *before* adding our own damage area, because
we want to only store the changes of this frame, not the whole history.

So do it in the same place Vulkan does it.

Fixes #1900

text: Ensure indexes are ordered properly

widget: Make gtk_widget_render() static

Merge branch 'menubutton-no-button' into 'master'

Menubutton: don't derive from GtkButton

See merge request GNOME/gtk!853

Adapt tests

We no longer have a clicked signal on GtkMenuButton.

Update all users for menubutton changes

a11y: Adapt to menu button changes

menubutton: Don't derive from GtkToggleButton

Make GtkMenuButton a widget that has a
toggle button, instead of deriving from it.

We give it icon-name and label properties,
to let people do what they expect to do
with menu buttons.

gl renderer: Render fallback nodes upside down

We stuff both gl-drawn and cairo-drawn textures into the same cache, so
we can't really assume that we need to draw any of them flipped or not.
Fix this by drawing fallback stuff upside down and then using
upside-down vertex data for everything.

Fixes #1897

gl renderer: Remove an outdated comment

gl renderer: Fix color matrix shader

We *just* computed a un-premultiplied color, maybe we should also use
it.

testsuite: Add a color matrix test case

Color matrix nodes with an identity matrix and no offset should still
produce the expected image.

gl renderer: Skip invisible shadows

gl renderer: Remove useless ops_offset calls

We add the shadow offset manually.

issue templates: reproducers should be written in C

I don't want to install a new set of bindings every time someone
attaches a reproducer to a bug. I also don't want to rewrite sait
reproducer in C every time just to eliminate the possibility of broken
bindings.

sidebarrow: Don't focus on click

This is sometimes important, e.g. when saving in the filechooser. We
don't want to move the focus out of the filename entry in that case.

widget-factory: Add icon-dropshadow class to large icon

Looks stupid without the shadow.

filechooserwidget: Remove unused gobject data

Merge branch 'fix-non-csd-menu-corners-gtk4' into 'master'

Adwaita: Don't round the non-csd menu corners (GTK4)

See merge request GNOME/gtk!835

Merge branch 'openbsd-pid_get_parenvt-v2' into 'master'

pid_get_parent: fix potential leak of kp

See merge request GNOME/gtk!844

pid_get_parent: fix potential leak of kp

Add clock freezes to the profile

Merge branch 'frame-profiling' into 'master'

always record a frame end time when profiling

See merge request GNOME/gtk!850

always record a frame end time when profiling

We need to keep this time, or we may end up with
negative frame intervals.

Merge branch 'frame-profiling' into 'master'

Redo frame profiling

See merge request GNOME/gtk!849

Redo frame profiling

We were adding incomplete frame timings to the
profile, which lead to occasional nonsense
numbers. Instead, only add timings to the profile
once we marked them as complete. This also
gives us an opportunity to add the presentation
time as a marker.

Updated Spanish translation

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!843

flattenlistmodel: Do proper check

If we want the model items to be listmodels, we don't need to do a ==
comparison. We need to do g_type_is_a(). Implementations of listmodels
are obviously fine.

profiler: Prefer CLOCK_MONOTONIC

This is what g_get_monotonic_time gives us,
after all, and we need to have the same clock
in all our profiler data.

query: Remove private pointer

filechooserwidget: Remove some unused struct members

accellabel: use a box layout

Instead of a GtkBox child widget.

Merge branch 'wip/carlosg/no-x11-core-events' into 'master'

Drop support for X11 core events

See merge request GNOME/gtk!846

gdk/x11: Remove XI1 remnant

This header is not referenced anywhere.

gdk/x11: Drop GdkDeviceManagerCore

Keep only the XInput2 device manager, which handles "modern" stuff
like scroll and touch.

gdk/x11: Make XInput2 a mandatory runtime dependency for x11

Besides requiring it at build time, require that the server the client
is running against exposes the XInput2 protocol. We no longer fallback
on a device manager for core events.

gdk/x11: Drop GDK_CORE_DEVICE_EVENTS environment variable

Support for core events will be discontinued, so this doesn't make sense
anymore.

gdk/x11: Remove gdk_disable_multidevice()

Core events will be discontinued, so this call is meaningless now.

gdk/x11: Make XInput2 a mandatory build time dependency for x11

XInput2 is more than a decade old already, and the input improvements
there (and in every other backend really) make it untenable to have
support for X11 core input events dragging things behind.

Try again to fix profiler setup

We were trying to store the profiler_id
in a struct that does not exist at the time.
Store it somewhere else.

Fix registration of profiler DBus api

profiling: Only use frame timings when we have them

Christian Hergert reported a crash here,
when testing the profiling code.

cellrendererpixbuf: Avoid creating expander textures...

... every frame. Create them once when setting the pixbuf instead.

cellrendererpixbuf: Remove an unused local variable

Merge branch 'openbsd-pid_get_parent' into 'master'

pid_get_parent: fix for OpenBSD

See merge request GNOME/gtk!841

pid_get_parent: fix for OpenBSD

This fixes a long standing bug in pid_get_parent on OpenBSD (which was mine
so... my fault). kp wasn't properly allocated and the function could return
random failures.

Merge branch 'modifier-events' into 'master'

key controller: Don't eat modifier events

Closes #1884

See merge request GNOME/gtk!839

key controller: Don't eat modifier events

The key controller was consuming key events
for modifier keys, for no entirely convincing
reason, which leads to problems when somebody
actually listens for those, such as the simple
input method does for C-S-u processing.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1884

Update POTFILES.in

Merge branch 'default-decoration-dialogs' into 'master'

dialog: Use default decoration for non-custom headerbars

See merge request GNOME/gtk!836

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!838

rendernodeparser: Move EOF test into parse_declarations()

This way, we don't have to duplicate code.

Plus, we can ensure to clear the erroneously parsed value that should
not be used.

rendernodeparser: Introduce clear_func

We want to be able to clear the already parsed contents, because we need
to do that when somebody parses the same property twice.

dialog: Use default decoration for non-custom headerbars

There are two ways GTK can add a headerbar to a dialog:

 - the dialog is constructed with the :use-header-bar property

 - all windows should use client-side decorations

In the first case, the headerbar is added by GtkDialog with no
dedicated style class, and in the latter by GtkWindow with the
"default-decoration" style.

As a result, dialogs with plain titlebars can end up with clearly
distinct and inconsistent styles.

To address this, allow headerbars to track whether they should use
the "default-decoration" style and enable it for dialogs.

https://gitlab.gnome.org/GNOME/gtk/merge_requests/836

about dialog: Fix page visibility

The handling of page visibility broke when the dialog
was ported to GtkStackPage in 52b83ac5530940bf17bc.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1877

Reinstate the activate-focus keybinding

Commit 3b62d9c027186fe0fddfb9 was a bit overzealous in
removing not just the public API for activating the focus
widget, but also the plumbing needed to make the keybinding
work.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1879

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!832

rendernodeparser: Default color to black

Nobody wants to render transparent text.

cssparser: Make gsk_renderer_consume_url() return a string

We don't want to return a GFile because GFile can't handle can't deal
with data: urls.
That makes the code a bit more complicated that doesn't deal with those
URLs, but it makes the other code actually work.

GtkCssImageUrl also now decodes data urls immediately instead of only at
the first load. So don't use data urls if you care about performance.

rendernodeparser: Use parser blocks

Instead of trying to do error handling manually, use the semicolon
blocks feature provided by GtkCssParser.

cssparser: Don't allow commit_token() on block EOF

When we're at the end of a block and gtk_css_parser_get_token() returns
NULL, gtk_css_parser_commit_token() still consumed the next token.

It does not anymore.

This does not affect the CSS parser, but it exposes issues with the
render parser, which previously just consumed too many closing } tokens
in the past.

testsuite: Stop using g_test_bug()

The calls used old bugzilla URLs and nobody cared about that.
So apparently they are very unused.

There's also a potential conflict between gitlab and bugzilla URLs and
what base bug to use there.

The old usages have been converted to comments.

node-editor: Report all errors in tooltip

Don't just report the first one.

node-editor: Don't remove all text tags when saving

testsuite: Don't die on the first error

Continue running the test, just mark it as a failure.

rendernodeparser: Parse images differently

Instead of encoding the raw data, encode the full image to a PNG.
And instead of stuffing that encoding into a string, use a full
data: url.
And then remove the width and height properties, because they're now
implicitly included in the data.
And then change the parser to match.
And because the parser now parses regular urls on top of data: urls, we
can now load any random file.

css: Add data URL parsing to images

css: Add gtk_css_data_url_parse()

This surprisingly decodes data URLs.

rendernode: Set EXTEND_PAD when drawing textures

Adwaita: Don't round the non-csd menu corners

Otherwise, the menu will have ugly black corners.

See https://gitlab.gnome.org/GNOME/gtk/issues/1661

Merge branch 'ci-master-fedora-30' into 'master'

CI: update docker image to Fedora 30

See merge request GNOME/gtk!802

CI: update docker image to fedora 30

So we don't have to build glib as a subproject.

settings: Update docs

We no longer create settings on demand.

css provider: Stop mentioning key themes

We no longer do this.

glrenderer: Ensure current context for debug stuff

rendernode: Parse repeat nodes

node-editor: Never scale nodes up

Scaling down is fine so one can see the nodes, but scaling up just
makes the nodes wrong for no reason at all.

Drop the public profiler api

I meant to do this before merging the profiler
support. Somehow the commit got lost.

Profiler: drop the public api